-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
userID: break out EID configuration to individual submodules #10229
Conversation
Nice! |
this is great! LGTM |
…10229) * move eids config to submodules * update EID logic * Fix tests
@@ -242,6 +242,59 @@ export const liveIntentIdSubmodule = { | |||
} | |||
|
|||
return { callback: result }; | |||
}, | |||
eids: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dgirardi I have a question related to changes in this PR: LiveIntent's user id module can also provide uid2s, but in case the uid2idSystem
is not part of the Prebid built, it will be impossible to convert them into the eids.
What should the intended usage pattern be in general? Is it "if you want an id of a certain type to be provided by a 'secondary' provider, you must have the 'main' provider too"? Or is it not the intention and LiveIntent's user id module should have a recipe for converting uid2s into eids?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@peixunzhang FYI
Type of change
Description of change
Closes #9786